Skip to content

refactor: ImageInstanceManager#mapImage#244

Merged
coding-jjun merged 2 commits intomainfrom
refactor-WISH-453
Mar 16, 2025
Merged

refactor: ImageInstanceManager#mapImage#244
coding-jjun merged 2 commits intomainfrom
refactor-WISH-453

Conversation

@ChoiWheatley
Copy link
Collaborator

PR 제목:

펀딩 및 이미지 매핑 로직 개선 및 리팩토링

PR 내용:

펀딩 목록 조회 및 이미지 매핑 로직을 개선하여 코드의 가독성을 높이고, 불필요한 변수 선언 및 중복 로직을 제거하였습니다.

주요 변경 사항:

  1. 펀딩 목록 조회 최적화

    • funding.fundUser 필터링 로직을 간소화.
    • 불필요한 friendIdsArray 변수 선언 제거.
    • friendIds 조회 시 userId를 재사용하여 코드 간결화.
  2. 이미지 매핑 개선

    • ImageInstanceManager#mapImage를 활용하여 네스팅된 엔티티에서도 이미지 매핑 가능하도록 개선.
    • leftJoinAndMapOne을 직접 사용하는 대신 mapImage를 호출하도록 변경.
  3. mapImages 메서드 개선

    • 엔티티 타입을 자동 감지하여 idFieldimgType을 동적으로 결정하도록 개선.
    • leftJoinAndMapMany를 활용하여 여러 이미지를 한 번에 매핑 가능하도록 변경.
    • defaultImgId가 존재할 경우 imgId를, 없을 경우 subIdimgType을 활용해 매핑.

기대 효과:

  • 코드 가독성 향상 및 유지보수성 증가.
  • 불필요한 변수 및 중복 코드 제거로 성능 개선.
  • ImageInstanceManager를 통해 이미지 매핑을 보다 일관성 있게 수행 가능.

검토 후 피드백 부탁드립니다! 🚀

Copy link
Owner

@coding-jjun coding-jjun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@coding-jjun coding-jjun merged commit e13b6a5 into main Mar 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants